Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add Tera:new_uninitialized (#677) #845

Closed
wants to merge 1 commit into from
Closed

feat: add Tera:new_uninitialized (#677) #845

wants to merge 1 commit into from

Conversation

seancroach
Copy link

@seancroach seancroach commented Jul 25, 2023

Exposes a new function that creates an instance of Tera without registering the default filters, testers, and functions. If this is merged, the version of tera in Cargo.toml will have to get its minor version incremented. No tests were added as no new code was created (except for a function declaration); I simply cut and paste the body from the Default implementation. I need this functionality and think no-ops as an overwrite are too hacky. I hope others can benefit from this, too.

Exposes a new function which creates an instance of Tera without
registering the default filters, testers, and functions. If this is
merged, the version of `tera` in `Cargo.toml` will have to get
its minor version incremented. No tests were added as no new
code was actually; I simply cut and paste the body from the
`Default` implementation. I need this functionality and thing no-ops
as an overwrite are too hacky. I hope others can benefit from this, too.
@seancroach seancroach changed the title feat: add Tera::new_unitialized (#677) feat: add Tera:new_uninitialized (#677) Jul 25, 2023
@Keats
Copy link
Owner

Keats commented Jul 27, 2023

I'm not a fan of the name but I can see the need hmm

@seancroach
Copy link
Author

That's totally fine. I made this rather quickly, and just made it... literal. Augment it however you'd like; I have no strong opinions on the matter.

@seancroach seancroach closed this by deleting the head repository Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants